Patch lodash template #64985

kobelb · 2020-05-01T01:15:52Z

No description provided.

legrego

This looks great!

legrego · 2020-05-05T15:06:14Z

test/harden/lodash.js

+  t.equal(global.whoops, undefined);
+  t.end();
+});
+


Do you think it's worthwhile to test lodash with the fourth guard parameter provided? In other words, testing that calling _.template as an iteree of _.map still works as expected? I only ask because there is special handling today to clear out the provided options:

https://github.com/lodash/lodash/blob/ddfd9b11a0126db2302cb70ec9973b66baec0975/lodash.js#L14776-L14778

I do... I need to give the guard parameter more attention. I came across it on Friday, grew frustrated, and moved on...

jbudz · 2020-05-06T16:41:16Z

I'm a little confused on the behavior of require-itm. Is this expected to cover nesting? kibana -> require(foo) -> require(lodash 3 not elastic patch). Can it interfere across major versions?

kobelb · 2020-05-06T16:55:47Z

I'm a little confused on the behavior of require-itm. Is this expected to cover nesting? kibana -> require(foo) -> require(lodash 3 not elastic patch). Can it interfere across major versions?

As far as I'm aware, yes. It will intercept all require/import's of a module regardless of the version. You've brought up a good point though, as there are usages throughout Kibana that use require('lodash/fp') not just require('lodash'). @watson, can you confirm this?

watson · 2020-05-11T07:13:23Z

@kobelb yes, RITM will intercept all require-calls throughout the current Node.js process - including require-calls made by dependencies.

Regarding scoping it to certain versions, I've done this previously in the Elastic APM Node.js Agent where I also use this module. Here I just require the package.json file of the module being required and look in the version field. I do this in the main hook-callback and then pass in the version along with the exports object to the patch, so each patch can choose to deal with it as they please. See here for implementation details: https://github.com/elastic/apm-agent-nodejs/blob/6a84486ccf8eb6a5b6e373ae06eabdcca5b66f7a/lib/instrumentation/index.js#L164-L181

Regarding lodash vs. lodash/fp, you should be able to simply add both to the array you pass in as the first argument to RITM.

watson · 2020-05-11T07:02:54Z

src/setup_node_env/harden.js

@@ -22,3 +22,7 @@ var hook = require('require-in-the-middle');
 hook(['child_process'], function(exports, name) {
  return require(`./patches/${name}`)(exports); // eslint-disable-line import/no-dynamic-require
 });
+
+hook(['lodash'], function(exports, name) {


You don't need to set up a new hook, instead you just add lodash to the array up above

Awesome, thanks Thomas!

legrego · 2020-06-29T11:59:46Z

@kobelb are you able to resurrect this? We'd love to include this work to unblock #69868 if possible

kobelb · 2020-06-30T14:40:25Z

@kobelb are you able to resurrect this? We'd love to include this work to unblock #69868 if possible

Yup! It definitely can be resurrected, there are no good reasons for why I let this PR go stale.

…late

… an iteratee

chore(NA): extend patches to lodash/fp and test for using template as an iteratee

mistic · 2020-07-02T14:32:57Z

I think the X-pack Jest suite failure will go away as soon as we merge the last upstream/master on that PR. I remember to see those failures a couple of days ago.

mistic · 2020-07-02T19:14:29Z

@elasticmachine merge upstream

kobelb · 2020-07-02T19:27:49Z

I created mistic#12 so that we can perform this patch as part of the lodash@4 upgrade. This "works", but it's somewhat fragile in regard to the way the patching work when _.template is used with _.map... With lodash@3, I couldn't use var isIterateeCall = require('lodash/_isIterateeCall'); directly, but when we're using lodash@4 I could.

* Applying changes from elastic#64985 * Using isIterateeCall, because it seems less brittle * Also patching `lodash/template` and `lodash/fp/template` * Reorganizing some files... * Revising comment

mistic · 2020-07-02T20:53:22Z

I've merged the PR and I'm letting the CI run. If that passes I'll merge the lodash upgrade PR! Thanks for your contribution here!

kibanamachine · 2020-07-02T20:55:05Z

💔 Build Failed

continuous-integration/kibana-ci/pull-request
Commit: 179198f
Pipeline Steps (look for red circles / failed steps)
Interpreting CI Failures

Failed CI Steps

Execute kibana-intake

Build metrics

✅ unchanged

History

💔 Build #58368 failed 5b3ba3e
💔 Build #58252 failed 5dde890
💔 Build #57552 failed 14c49e6
💚 Build #44697 succeeded 5b44865

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

* chore(NA): upgrade oss to lodash4 chore(NA): migrate cli, cli_plugin, cli_keystore, dev, test_utils and apm src script to lodash4 chore(NA): missing file for cli plugin chore(NA): add src core chore(NA): es archiver and fixtures chore(NA): try to fix functional test failure chore(NA): migrate src/legacy entirely to lodash4 except src/legacy/core_plugins chore(NA): move legacy core plugins to lodash4 chore(NA): upgrade optimize to lodash4 chore(NA): upgrade to lodash4 on advanced_settings, charts, console and dashboard chore(NA): migrate to lodash4 on dev_tools, discover, embeddable, es_ui)shared, expressions, home plugins chore(NA): upgrade data plugin to lodash4 chore(NA): upgrade usage_collection, ui_actions, tile_map, telemtry, share, saved_objects, saved_objects_management, region_map and navigation to lodash4 chore(NA): missing data upgrades to lodash4 Revert "chore(NA): upgrade usage_collection, ui_actions, tile_map, telemtry, share, saved_objects, saved_objects_management, region_map and navigation to lodash4" This reverts commit 137055c5fed2fc52bb26547e0bc1ad2e3d4fe309. Revert "Revert "chore(NA): upgrade usage_collection, ui_actions, tile_map, telemtry, share, saved_objects, saved_objects_management, region_map and navigation to lodash4"" This reverts commit f7e73688782998513d9fb6d7e8f0765e9beb28d1. Revert "chore(NA): missing data upgrades to lodash4" This reverts commit 92b85bf947a89bfc70cc4052738a6b2128ffb076. Revert "chore(NA): upgrade data plugin to lodash4" This reverts commit 88fdb075ee1e26c4ac979b6681d8a2b002df74c6. chore(NA): upgrade idx_pattern_mgt, input_control_vis, inspector, kbn_legacy, kbn_react, kbn_usage_collections, kbn_utils, management and maps_legacy to lodash4 chore(NA): map src plugin data to lodash3 chore(NA): missing lodash.clonedeep dep chore(NA): change packages kbn-config-schema deps chore(NA): update renovate config chore(NA): upgrade vis_type plugins to lodash4 chore(NA): move vis_type_vislib to lodash3 chore(NA): update visualizations and visualize to lodash4 chore(NA): remove lodash 3 types from src and move test to lodash4 chore(NA): move home, usage_collection and management to lodash 3 Revert "chore(NA): move home, usage_collection and management to lodash 3" This reverts commit f86e8585f02d21550746569af54215b076a79a3d. chore(NA): move kibana_legacy, saved_objects saved_objects_management into lodash3 chore(NA): update x-pack test to mock lodash4 Revert "chore(NA): move kibana_legacy, saved_objects saved_objects_management into lodash3" This reverts commit 2d10fe450533e1b36db21d99cfae3ce996a244e0. * chore(NA): move x-pack and packages to lodash 4 * chore(NA): remove mention to lodash from main package.json * chore(NA): remove helper alias for lodash4 and make it the default lodash * chore(NA): fix last failing types in the repo * chore(NA): fix public api * chore(NA): fix types for agg_row.tsx * chore(NA): fix increment of optimizer modules in the rollup plugin * chore(NA): migrate `src/core/public/http/fetch.ts` (#5) * omit undefined query props * just remove merge usage * fix types * chore(NA): fixes for feedback from apm team * chore(NA): recover old behaviour on apm LoadingIndeicatorContext.tsx * chore(NA): fixes for feedback from watson * Platform lodash4 tweaks (#6) * chore(NA): fix types and behaviour on src/core/server/elasticsearch/errors.ts * Canvas fixes for lodash upgrade * [APM] Adds unit test for APM service maps transform (#7) * Adds a snapshot unit test for getConnections and rearranges some code to make testing easier * reverts `ArrayList` back to `String[]` in the painless script within `fetch_service_paths_from_trace_ids.ts` * chore(NA): update yarn.lock * chore(NA): remove any and use a real type for alerts task runner Co-authored-by: Gidi Meir Morris <[email protected]> * chore(NA): used named import for triggers_actions_ui file * chore(NA): fix eslint * chore(NA): fix types * Delete most uptime lodash references. * Simplify. Clean up types. * [Uptime] Delete most uptime lodash references (#8) * Delete most uptime lodash references. * Simplify. Clean up types. * chore(NA): add eslint rule to avoid using lodash3 * chore(NA): apply changes on feedback from es-ui team * fix some types (#9) * Clean up some expressions types. * chore(NA): missing ts-expect-error statements * Upgrade lodash 4 vislib (#11) * replace lodash 3 with lodash 4 on vislib plugin * Further changes * further replacement of lodash3 to 4 * further work on upgrading to lodash 4 * final changes to update lodash * chore(NA): upgrade data plugin to lodash4 chore(NA): upgrade data plugin public to lodash4 chore(NA): fix typecheck task chore(NA): fix agg_config with hasIn chore(NA): assign to assignIn and has to hasIn chore(NA): upgrade data plugin server to lodash4 chore(NA): new signature for core api fix(NA): match behaviour between lodash3 and lodash4 for set in search_source * chore(NA): remove lodash3 completely from the repo * chore(NA): fix x-pack/test/api_integration/apis/metrics_ui/snapshot.ts missing content * chore(NA): fix lodash usage on apm * chore(NA): fix typecheck for maps * Patch lodash template (#12) * Applying changes from #64985 * Using isIterateeCall, because it seems less brittle * Also patching `lodash/template` and `lodash/fp/template` * Reorganizing some files... * Revising comment * Ends up `_` is a function also... I hate JavaScript Co-authored-by: Pierre Gayvallet <[email protected]> Co-authored-by: Josh Dover <[email protected]> Co-authored-by: Clint Andrew Hall <[email protected]> Co-authored-by: Oliver Gupte <[email protected]> Co-authored-by: Elastic Machine <[email protected]> Co-authored-by: Gidi Meir Morris <[email protected]> Co-authored-by: Justin Kambic <[email protected]> Co-authored-by: Stratoula Kalafateli <[email protected]> Co-authored-by: Luke Elmers <[email protected]> Co-authored-by: Brandon Kobel <[email protected]> Co-authored-by: kobelb <[email protected]>

* chore(NA): upgrade oss to lodash4 chore(NA): migrate cli, cli_plugin, cli_keystore, dev, test_utils and apm src script to lodash4 chore(NA): missing file for cli plugin chore(NA): add src core chore(NA): es archiver and fixtures chore(NA): try to fix functional test failure chore(NA): migrate src/legacy entirely to lodash4 except src/legacy/core_plugins chore(NA): move legacy core plugins to lodash4 chore(NA): upgrade optimize to lodash4 chore(NA): upgrade to lodash4 on advanced_settings, charts, console and dashboard chore(NA): migrate to lodash4 on dev_tools, discover, embeddable, es_ui)shared, expressions, home plugins chore(NA): upgrade data plugin to lodash4 chore(NA): upgrade usage_collection, ui_actions, tile_map, telemtry, share, saved_objects, saved_objects_management, region_map and navigation to lodash4 chore(NA): missing data upgrades to lodash4 Revert "chore(NA): upgrade usage_collection, ui_actions, tile_map, telemtry, share, saved_objects, saved_objects_management, region_map and navigation to lodash4" This reverts commit 137055c5fed2fc52bb26547e0bc1ad2e3d4fe309. Revert "Revert "chore(NA): upgrade usage_collection, ui_actions, tile_map, telemtry, share, saved_objects, saved_objects_management, region_map and navigation to lodash4"" This reverts commit f7e73688782998513d9fb6d7e8f0765e9beb28d1. Revert "chore(NA): missing data upgrades to lodash4" This reverts commit 92b85bf947a89bfc70cc4052738a6b2128ffb076. Revert "chore(NA): upgrade data plugin to lodash4" This reverts commit 88fdb075ee1e26c4ac979b6681d8a2b002df74c6. chore(NA): upgrade idx_pattern_mgt, input_control_vis, inspector, kbn_legacy, kbn_react, kbn_usage_collections, kbn_utils, management and maps_legacy to lodash4 chore(NA): map src plugin data to lodash3 chore(NA): missing lodash.clonedeep dep chore(NA): change packages kbn-config-schema deps chore(NA): update renovate config chore(NA): upgrade vis_type plugins to lodash4 chore(NA): move vis_type_vislib to lodash3 chore(NA): update visualizations and visualize to lodash4 chore(NA): remove lodash 3 types from src and move test to lodash4 chore(NA): move home, usage_collection and management to lodash 3 Revert "chore(NA): move home, usage_collection and management to lodash 3" This reverts commit f86e8585f02d21550746569af54215b076a79a3d. chore(NA): move kibana_legacy, saved_objects saved_objects_management into lodash3 chore(NA): update x-pack test to mock lodash4 Revert "chore(NA): move kibana_legacy, saved_objects saved_objects_management into lodash3" This reverts commit 2d10fe450533e1b36db21d99cfae3ce996a244e0. * chore(NA): move x-pack and packages to lodash 4 * chore(NA): remove mention to lodash from main package.json * chore(NA): remove helper alias for lodash4 and make it the default lodash * chore(NA): fix last failing types in the repo * chore(NA): fix public api * chore(NA): fix types for agg_row.tsx * chore(NA): fix increment of optimizer modules in the rollup plugin * chore(NA): migrate `src/core/public/http/fetch.ts` (#5) * omit undefined query props * just remove merge usage * fix types * chore(NA): fixes for feedback from apm team * chore(NA): recover old behaviour on apm LoadingIndeicatorContext.tsx * chore(NA): fixes for feedback from watson * Platform lodash4 tweaks (#6) * chore(NA): fix types and behaviour on src/core/server/elasticsearch/errors.ts * Canvas fixes for lodash upgrade * [APM] Adds unit test for APM service maps transform (#7) * Adds a snapshot unit test for getConnections and rearranges some code to make testing easier * reverts `ArrayList` back to `String[]` in the painless script within `fetch_service_paths_from_trace_ids.ts` * chore(NA): update yarn.lock * chore(NA): remove any and use a real type for alerts task runner Co-authored-by: Gidi Meir Morris <[email protected]> * chore(NA): used named import for triggers_actions_ui file * chore(NA): fix eslint * chore(NA): fix types * Delete most uptime lodash references. * Simplify. Clean up types. * [Uptime] Delete most uptime lodash references (#8) * Delete most uptime lodash references. * Simplify. Clean up types. * chore(NA): add eslint rule to avoid using lodash3 * chore(NA): apply changes on feedback from es-ui team * fix some types (#9) * Clean up some expressions types. * chore(NA): missing ts-expect-error statements * Upgrade lodash 4 vislib (#11) * replace lodash 3 with lodash 4 on vislib plugin * Further changes * further replacement of lodash3 to 4 * further work on upgrading to lodash 4 * final changes to update lodash * chore(NA): upgrade data plugin to lodash4 chore(NA): upgrade data plugin public to lodash4 chore(NA): fix typecheck task chore(NA): fix agg_config with hasIn chore(NA): assign to assignIn and has to hasIn chore(NA): upgrade data plugin server to lodash4 chore(NA): new signature for core api fix(NA): match behaviour between lodash3 and lodash4 for set in search_source * chore(NA): remove lodash3 completely from the repo * chore(NA): fix x-pack/test/api_integration/apis/metrics_ui/snapshot.ts missing content * chore(NA): fix lodash usage on apm * chore(NA): fix typecheck for maps * Patch lodash template (#12) * Applying changes from elastic#64985 * Using isIterateeCall, because it seems less brittle * Also patching `lodash/template` and `lodash/fp/template` * Reorganizing some files... * Revising comment * Ends up `_` is a function also... I hate JavaScript Co-authored-by: Pierre Gayvallet <[email protected]> Co-authored-by: Josh Dover <[email protected]> Co-authored-by: Clint Andrew Hall <[email protected]> Co-authored-by: Oliver Gupte <[email protected]> Co-authored-by: Elastic Machine <[email protected]> Co-authored-by: Gidi Meir Morris <[email protected]> Co-authored-by: Justin Kambic <[email protected]> Co-authored-by: Stratoula Kalafateli <[email protected]> Co-authored-by: Luke Elmers <[email protected]> Co-authored-by: Brandon Kobel <[email protected]> Co-authored-by: kobelb <[email protected]> # Conflicts: # renovate.json5 # x-pack/plugins/apm/public/services/rest/observability_dashboard.ts # x-pack/plugins/canvas/common/lib/pivot_object_array.ts # x-pack/plugins/canvas/public/components/workpad_templates/workpad_templates.js # x-pack/plugins/canvas/server/collectors/custom_element_collector.ts # x-pack/plugins/canvas/server/collectors/workpad_collector.ts

* chore(NA): upgrade to lodash@4 (#69868) * chore(NA): upgrade oss to lodash4 chore(NA): migrate cli, cli_plugin, cli_keystore, dev, test_utils and apm src script to lodash4 chore(NA): missing file for cli plugin chore(NA): add src core chore(NA): es archiver and fixtures chore(NA): try to fix functional test failure chore(NA): migrate src/legacy entirely to lodash4 except src/legacy/core_plugins chore(NA): move legacy core plugins to lodash4 chore(NA): upgrade optimize to lodash4 chore(NA): upgrade to lodash4 on advanced_settings, charts, console and dashboard chore(NA): migrate to lodash4 on dev_tools, discover, embeddable, es_ui)shared, expressions, home plugins chore(NA): upgrade data plugin to lodash4 chore(NA): upgrade usage_collection, ui_actions, tile_map, telemtry, share, saved_objects, saved_objects_management, region_map and navigation to lodash4 chore(NA): missing data upgrades to lodash4 Revert "chore(NA): upgrade usage_collection, ui_actions, tile_map, telemtry, share, saved_objects, saved_objects_management, region_map and navigation to lodash4" This reverts commit 137055c5fed2fc52bb26547e0bc1ad2e3d4fe309. Revert "Revert "chore(NA): upgrade usage_collection, ui_actions, tile_map, telemtry, share, saved_objects, saved_objects_management, region_map and navigation to lodash4"" This reverts commit f7e73688782998513d9fb6d7e8f0765e9beb28d1. Revert "chore(NA): missing data upgrades to lodash4" This reverts commit 92b85bf947a89bfc70cc4052738a6b2128ffb076. Revert "chore(NA): upgrade data plugin to lodash4" This reverts commit 88fdb075ee1e26c4ac979b6681d8a2b002df74c6. chore(NA): upgrade idx_pattern_mgt, input_control_vis, inspector, kbn_legacy, kbn_react, kbn_usage_collections, kbn_utils, management and maps_legacy to lodash4 chore(NA): map src plugin data to lodash3 chore(NA): missing lodash.clonedeep dep chore(NA): change packages kbn-config-schema deps chore(NA): update renovate config chore(NA): upgrade vis_type plugins to lodash4 chore(NA): move vis_type_vislib to lodash3 chore(NA): update visualizations and visualize to lodash4 chore(NA): remove lodash 3 types from src and move test to lodash4 chore(NA): move home, usage_collection and management to lodash 3 Revert "chore(NA): move home, usage_collection and management to lodash 3" This reverts commit f86e8585f02d21550746569af54215b076a79a3d. chore(NA): move kibana_legacy, saved_objects saved_objects_management into lodash3 chore(NA): update x-pack test to mock lodash4 Revert "chore(NA): move kibana_legacy, saved_objects saved_objects_management into lodash3" This reverts commit 2d10fe450533e1b36db21d99cfae3ce996a244e0. * chore(NA): move x-pack and packages to lodash 4 * chore(NA): remove mention to lodash from main package.json * chore(NA): remove helper alias for lodash4 and make it the default lodash * chore(NA): fix last failing types in the repo * chore(NA): fix public api * chore(NA): fix types for agg_row.tsx * chore(NA): fix increment of optimizer modules in the rollup plugin * chore(NA): migrate `src/core/public/http/fetch.ts` (#5) * omit undefined query props * just remove merge usage * fix types * chore(NA): fixes for feedback from apm team * chore(NA): recover old behaviour on apm LoadingIndeicatorContext.tsx * chore(NA): fixes for feedback from watson * Platform lodash4 tweaks (#6) * chore(NA): fix types and behaviour on src/core/server/elasticsearch/errors.ts * Canvas fixes for lodash upgrade * [APM] Adds unit test for APM service maps transform (#7) * Adds a snapshot unit test for getConnections and rearranges some code to make testing easier * reverts `ArrayList` back to `String[]` in the painless script within `fetch_service_paths_from_trace_ids.ts` * chore(NA): update yarn.lock * chore(NA): remove any and use a real type for alerts task runner Co-authored-by: Gidi Meir Morris <[email protected]> * chore(NA): used named import for triggers_actions_ui file * chore(NA): fix eslint * chore(NA): fix types * Delete most uptime lodash references. * Simplify. Clean up types. * [Uptime] Delete most uptime lodash references (#8) * Delete most uptime lodash references. * Simplify. Clean up types. * chore(NA): add eslint rule to avoid using lodash3 * chore(NA): apply changes on feedback from es-ui team * fix some types (#9) * Clean up some expressions types. * chore(NA): missing ts-expect-error statements * Upgrade lodash 4 vislib (#11) * replace lodash 3 with lodash 4 on vislib plugin * Further changes * further replacement of lodash3 to 4 * further work on upgrading to lodash 4 * final changes to update lodash * chore(NA): upgrade data plugin to lodash4 chore(NA): upgrade data plugin public to lodash4 chore(NA): fix typecheck task chore(NA): fix agg_config with hasIn chore(NA): assign to assignIn and has to hasIn chore(NA): upgrade data plugin server to lodash4 chore(NA): new signature for core api fix(NA): match behaviour between lodash3 and lodash4 for set in search_source * chore(NA): remove lodash3 completely from the repo * chore(NA): fix x-pack/test/api_integration/apis/metrics_ui/snapshot.ts missing content * chore(NA): fix lodash usage on apm * chore(NA): fix typecheck for maps * Patch lodash template (#12) * Applying changes from #64985 * Using isIterateeCall, because it seems less brittle * Also patching `lodash/template` and `lodash/fp/template` * Reorganizing some files... * Revising comment * Ends up `_` is a function also... I hate JavaScript Co-authored-by: Pierre Gayvallet <[email protected]> Co-authored-by: Josh Dover <[email protected]> Co-authored-by: Clint Andrew Hall <[email protected]> Co-authored-by: Oliver Gupte <[email protected]> Co-authored-by: Elastic Machine <[email protected]> Co-authored-by: Gidi Meir Morris <[email protected]> Co-authored-by: Justin Kambic <[email protected]> Co-authored-by: Stratoula Kalafateli <[email protected]> Co-authored-by: Luke Elmers <[email protected]> Co-authored-by: Brandon Kobel <[email protected]> Co-authored-by: kobelb <[email protected]> # Conflicts: # renovate.json5 # x-pack/plugins/apm/public/services/rest/observability_dashboard.ts # x-pack/plugins/canvas/common/lib/pivot_object_array.ts # x-pack/plugins/canvas/public/components/workpad_templates/workpad_templates.js # x-pack/plugins/canvas/server/collectors/custom_element_collector.ts # x-pack/plugins/canvas/server/collectors/workpad_collector.ts * chore(NA): updated kbn-pm dist file

kobelb added 2 commits April 30, 2020 14:07

Patching lodash's template

8a35ccc

And now some tests!

c7ed319

kobelb added the release_note:skip Skip the PR/issue when compiling release notes label May 1, 2020

kobelb requested review from watson and legrego May 1, 2020 01:15

kobelb requested a review from a team as a code owner May 1, 2020 01:15

Removing errant console.log

5b44865

legrego reviewed May 5, 2020

View reviewed changes

jbudz approved these changes May 6, 2020

View reviewed changes

watson suggested changes May 11, 2020

View reviewed changes

Merge remote-tracking branch 'upstream/master' into patch-lodash-temp…

14c49e6

…late

mistic mentioned this pull request Jun 30, 2020

chore(NA): upgrade to lodash@4 #69868

Merged

3 tasks

chore(NA): extend patches to lodash/fp and test for using template as…

a0bc325

… an iteratee

mistic mentioned this pull request Jul 1, 2020

chore(NA): extend patches to lodash/fp and test for using template as an iteratee kobelb/kibana#13

Merged

Merge pull request #13 from mistic/patch-lodash-template

5dde890

chore(NA): extend patches to lodash/fp and test for using template as an iteratee

Separate patch for 'lodash/fp'

5b3ba3e

Merge branch 'master' into patch-lodash-template

179198f

kobelb added a commit to kobelb/kibana that referenced this pull request Jul 2, 2020

Applying changes from elastic#64985

75320b9

kobelb mentioned this pull request Jul 2, 2020

Patch lodash template mistic/kibana#12

Merged

kobelb closed this Sep 22, 2020

kobelb deleted the patch-lodash-template branch September 22, 2020 13:42

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Patch lodash template #64985

Patch lodash template #64985

kobelb commented May 1, 2020

legrego left a comment

legrego May 5, 2020

kobelb May 5, 2020

jbudz commented May 6, 2020

kobelb commented May 6, 2020

watson commented May 11, 2020

watson May 11, 2020

kobelb May 11, 2020

legrego commented Jun 29, 2020

kobelb commented Jun 30, 2020

mistic commented Jul 2, 2020

mistic commented Jul 2, 2020

kobelb commented Jul 2, 2020

mistic commented Jul 2, 2020

kibanamachine commented Jul 2, 2020

Patch lodash template #64985

Patch lodash template #64985

Conversation

kobelb commented May 1, 2020

legrego left a comment

Choose a reason for hiding this comment

legrego May 5, 2020

Choose a reason for hiding this comment

kobelb May 5, 2020

Choose a reason for hiding this comment

jbudz commented May 6, 2020

kobelb commented May 6, 2020

watson commented May 11, 2020

watson May 11, 2020

Choose a reason for hiding this comment

kobelb May 11, 2020

Choose a reason for hiding this comment

legrego commented Jun 29, 2020

kobelb commented Jun 30, 2020

mistic commented Jul 2, 2020

mistic commented Jul 2, 2020

kobelb commented Jul 2, 2020

mistic commented Jul 2, 2020

kibanamachine commented Jul 2, 2020

💔 Build Failed

Failed CI Steps

Build metrics

History